Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add: placeholderWidget parameter #97

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

LeGoffMael
Copy link

@LeGoffMael LeGoffMael commented Dec 16, 2022

  • new placeholderWidget parameter to display a widget when the image is null
  • new onLoading function parameter to be aware when the crop area is ready to be used

also fixed an issue where switching image causes an exception (#85)

════════ Exception caught by image resource service ════════════════════════════
The following assertion was thrown by a synchronously-called image listener:
Cannot get size during build.

The size of this render object has not yet been determined because the framework is still in the process of building widgets, which means the render tree for this frame has not yet been determined. The size getter should only be called from paint callbacks or interaction event handlers (e.g. gesture callbacks).

example

here is a video showing how a thumbnail and a loader can be used as placeholder

RPReplay_Final1671184314.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant